Repository navigation
Fix full-deck Marp overflow and presenter identity - #17
Merged
Merged
Conversation
Preserve all 53 slides while fixing dense-slide spacing, bounding tall diagrams, and revealing the full observability artwork. Complete the canonical presenter identity and PDF metadata without changing talk content, demo code, notes, links, assets, or publishing workflows. Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Full-deck review
Reviewed commit:
d005c6b05ad6d6042d943f5e3ce375def1ad3d88.Used the installed
marp-slide-reviewscripts directly because the skill loader did not expose the skill. Every baseline PNG was inspected at 1280x720; final coverage is 41 pixel-identical slides plus 12 reinspected slides. Compared PDF pages 1, 2, 5, 8, 18, 19, 20, 27, 36, 37, 40, and 53 with the PNG output; all 53 PDF pages pass text-boundary checks.The installed overflow check passes at its default 2px tolerance. An additional asset-resolved HTML audit reports zero overflow, boundary violations, footer collisions, missing images, font failures, or runtime/network errors. Exact identity matches are 4 role / 4 affiliation in the deck (including metadata), plus 1 / 1 in README; the PDF has three visible identity pairs and the canonical author metadata.
Remaining source-image limitations
Existing raster diagram fine print remains small, notably on 5 and 19; 37 remains soft when enlarged from its 622x366 source. All embedded editable models are preserved. The tiny, cropped code in slide 8's stock-photo background is decorative, not the talk's Dockerfile, and has no editable source here.
Render evidence is retained outside the repository. No sample apps, security scanners, cloud deployments, model evaluations, or subagents were run; normal CI is unchanged.